Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Créer le composant des autres moyens de connexion/inscription (PIX-14159) #10275

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Oct 8, 2024

🦄 Problème

Nous avons besoin d'un nouveau composant pour pouvoir choisir les autres moyens de connexion/inscription

image-20240904-093826

image-20240904-093933

🤖 Proposition

  1. Faire que l'API renvoie aussi le slug de chaque OIDC Provider
  2. Faire évoluer le service oidcIdentityProviders pour qu’il fournisse les nouveaux getters featuredIdentityProvider et hasOtherIdentityProviders
  3. Ajouter un composant GJS s'appuyant sur le service oidcIdentityProviders pour afficher l’interface utilisateur souhaitée

🌈 Remarques

RAS

💯 Pour tester

On ne peut tester cela qu'en local car il n'y a pas de SSO configuré sur les RA.

  • Dans le fichier .env de l'api (déjà fait pour la RA), ajouter:
FT_NEW_AUTHENTICATION_DESIGN_ENABLED=true
  • Aller sur Pix App en version .fr et vérifier que la nouvelle mire de connexion s'affiche correctement (formulaire de connexion uniquement) avec les 2 boutons Continuer avec France Travail et Choisir une autre organisation

  • Aller sur Pix App en version .org

    • vérifier que la nouvelle mire de connexion s'affiche correctement (formulaire de connexion uniquement) avec 1 seul bouton Continuer avec Fédération Wallonie-Bruxelles et pas de bouton Choisir une autre organisation
    • Le texte de la page doit changer en fonction de la langue choisie avec le Language Switcher

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@lego-technix lego-technix marked this pull request as ready for review October 8, 2024 17:26
@lego-technix lego-technix requested a review from a team as a code owner October 8, 2024 17:26
@lego-technix lego-technix force-pushed the pix-14159-new-component-authentication-other-means branch 3 times, most recently from fdf80da to bddf0bd Compare October 9, 2024 10:02
@lego-technix lego-technix force-pushed the pix-14159-new-component-authentication-other-means branch 3 times, most recently from fe1425f to 4742606 Compare October 9, 2024 17:05
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il manque les icônes de boutons FT et FWB sur les boutons featured. Voir ce qui est indiqué pour le bouton FT dans le ticket:

mettre l’icône de France Travail : respecter les exigences graphiques indiquées dans ce lien : https://francetravail.io/produits-partages/documentation/5-savoir-dispositif-france-travail-connect/maquettes-charte

Il manque également l'icône de chevron à droite du bouton.

De plus, après vérification de la maquette, il semble que les boutons (de la maquette) ne respectent aucun style du design system. Il faudrait voir avec Pierre si c'est volontaire ou bien une erreur. Si c'est volontaire, il faut qu'on voit avec lui le style des différents états du bouton (hover, focus...) dans ce nouveau style.

@lego-technix lego-technix force-pushed the pix-14159-new-component-authentication-other-means branch 2 times, most recently from 7bbb78a to 39984ae Compare October 10, 2024 18:34
@lego-technix
Copy link
Contributor Author

Il manque les icônes de boutons FT et FWB sur les boutons featured. Voir ce qui est indiqué pour le bouton FT dans le ticket:

Il manque également l'icône de chevron à droite du bouton.

De plus, après vérification de la maquette, il semble que les boutons (de la maquette) ne respectent aucun style du design system. Il faudrait voir avec Pierre si c'est volontaire ou bien une erreur. Si c'est volontaire, il faut qu'on voit avec lui le style des différents états du bouton (hover, focus...) dans ce nouveau style.

Tous ces points ont été traités au mieux.

@lego-technix lego-technix force-pushed the pix-14159-new-component-authentication-other-means branch from 39984ae to 434311a Compare October 10, 2024 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants